Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Removed the dependency on Json.Net and stopped flatting and unflattin… #1

Merged
merged 3 commits into from
Dec 4, 2024
Merged

Removed the dependency on Json.Net and stopped flatting and unflattin… #1

merged 3 commits into from
Dec 4, 2024

Conversation

t11omas
Copy link
Contributor

@t11omas t11omas commented Dec 4, 2024

…g the Json

Now it just finds the value using JsonPath and generates a JsonPatch to update the value with a redacted value

Thomas Anderson added 3 commits December 4, 2024 08:01
Removed the dependency on Json.Net and stopped flatting and unflattin…
d9c114f 
…g the Json

Now it just finds the value using JsonPath and generates a JsonPatch to update the value with a redacted value
Updated path on test
8d1e544
Added call to start job
131ca6d
@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 4, 2024

Code Coverage

Package Line Rate Branch Rate Health
Btms.Business 87% 74% ✔
Btms.Types.Gvms.Mapping 38% 26% ❌
Btms.Metrics 82% 0% ✔
Btms.Types.Gvms 67% 100% ➖
Btms.Types.Alvs.Mapping 74% 0% ➖
Btms.Types.Ipaffs.Mapping 73% 40% ➖
TestDataGenerator 54% 47% ❌
Btms.Azure 70% 75% ➖
Btms.BlobService 31% 17% ❌
Btms.Backend.Data 66% 62% ➖
Btms.Consumers 70% 35% ➖
Btms.Common 79% 17% ➖
Btms.Model 91% 92% ✔
Btms.SyncJob 81% 71% ✔
Btms.SensitiveData 80% 81% ➖
Btms.Types.Ipaffs 94% 50% ✔
Btms.Types.Alvs.Mapping.V1.Tests 0% 0% ❌
Btms.Types.Alvs 83% 0% ✔
Btms.Emf 11% 6% ❌
Btms.Backend 59% 60% ❌
Btms.Analytics 80% 65% ✔
Summary 73% (5293 / 7238) 47% (632 / 1346) ➖

@craigedmunds craigedmunds merged commit 9aa001c into main Dec 4, 2024
1 check passed
@craigedmunds craigedmunds deleted the feature/cdms-165 branch December 4, 2024 08:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@t11omas @craigedmunds